Skip to content

Conversation

@OttoAllmendinger
Copy link
Contributor

This PR includes several improvements to the abstract-utxo package:

  • Refactor recovery tests to run with multiple script types simultaneously
    instead of separate runs for each type, improving test efficiency and
    enabling testing of script type combinations
  • Optimize test setup by precalculating keys instead of recalculating for
    each test run, reducing test discovery time from 10s to 6s
  • Wrap recovery tests in describe blocks for better clarity and improved
    VSCode test extension UX
  • Fix fixture path handling by removing special case for "tbtcbgsig"

Related to: BG-62732, BTC-2732

OttoAllmendinger and others added 5 commits November 13, 2025 14:25
Precalculate keys instead of recalculating them for each test run,
reducing test discovery time from 10s to 6s.

Issue: BG-62732

Co-authored-by: llm-git <[email protected]>
Remove special handling for "tbtcbgsig" in fixture path logic.

Issue: BTC-2732

Co-authored-by: llm-git <[email protected]>
…ipt types

Refactor the backup key recovery tests to run with multiple script types
at once instead of running separately for each script type. This makes
the tests more efficient and allows testing combinations of script types.

Issue: BTC-2732

Co-authored-by: llm-git <[email protected]>
Add utility to normalize bitcoin testnet variants to 'tbtc' for fixture
generation, ensuring consistent test fixtures across different testnet
environments.

Issue: BTC-2732

Co-authored-by: llm-git <[email protected]>
Improves UX with VSCode test extension.

Issue: BTC-2732

Co-authored-by: llm-git <[email protected]>
@OttoAllmendinger OttoAllmendinger force-pushed the BTC-2732.backupKeyRecoverySpeedup branch from ec11602 to 996ab3b Compare November 13, 2025 13:45
@OttoAllmendinger OttoAllmendinger marked this pull request as ready for review November 13, 2025 14:12
@OttoAllmendinger OttoAllmendinger requested a review from a team as a code owner November 13, 2025 14:12
@OttoAllmendinger OttoAllmendinger merged commit 046ebdf into master Nov 13, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants